Skip to content

Conversation

@yellowred
Copy link
Contributor

@yellowred yellowred commented Mar 18, 2025

Towards LPT-225

@yellowred yellowred requested review from a team and JasonCWang and removed request for a team March 18, 2025 01:18
Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

entity(id: $entity_id) {
... on Account {
transactions(, first: $first, after: $after, types: $types, after_date: $after_date, before_date: $before_date, bitcoin_network: $bitcoin_network, lightning_node_id: $lightning_node_id, statuses: $statuses, exclude_failures: $exclude_failures) {
transactions(, first: $first, after: $after, types: $types, after_date: $after_date, before_date: $before_date, bitcoin_network: $bitcoin_network, lightning_node_id: $lightning_node_id, statuses: $statuses, exclude_failures: $exclude_failures, max_amount: $max_amount, min_amount: $min_amount) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There appears to be a syntax error in the GraphQL query - an extra comma exists after the opening parenthesis in the transactions function call. This should be removed to prevent GraphQL parsing errors.

// Incorrect:
transactions(, first: $first, after: $after, ...)

// Should be:
transactions(first: $first, after: $after, ...)

The same issue appears in other queries in this file as well (payment_requests, withdrawal_requests). These extra commas should be removed for proper GraphQL syntax.

Suggested change
transactions(, first: $first, after: $after, types: $types, after_date: $after_date, before_date: $before_date, bitcoin_network: $bitcoin_network, lightning_node_id: $lightning_node_id, statuses: $statuses, exclude_failures: $exclude_failures, max_amount: $max_amount, min_amount: $min_amount) {
transactions(first: $first, after: $after, types: $types, after_date: $after_date, before_date: $before_date, bitcoin_network: $bitcoin_network, lightning_node_id: $lightning_node_id, statuses: $statuses, exclude_failures: $exclude_failures, max_amount: $max_amount, min_amount: $min_amount) {

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.

@@ -1,5 +1,6 @@
// Copyright ©, 2023-present, Lightspark Group, Inc. - All Rights Reserved
use crate::objects::connection::Connection;
use crate::objects::lightspark_node::LightsparkNode;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this unused import for LightsparkNode. It's not being used in this file and is causing a compiler warning.

Spotted by Diamond (based on CI logs)

Is this helpful? React 👍 or 👎 to let us know.

@yellowred
Copy link
Contributor Author

yellowred commented Mar 21, 2025

Too many errors, solved my problem using Sparkcore signing for now.

@yellowred yellowred closed this Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants